Skip to content

chore: use dyn InspectorExt in Backend#8919

Merged
DaniPopes merged 1 commit intomasterfrom
dani/evm-dyn-inspector-1
Sep 21, 2024
Merged

chore: use dyn InspectorExt in Backend#8919
DaniPopes merged 1 commit intomasterfrom
dani/evm-dyn-inspector-1

Conversation

@DaniPopes
Copy link
Copy Markdown
Member

Gets rid of revm monomorphization of NoopInspector (1/6 in forge)

@DaniPopes DaniPopes merged commit 1f9c77a into master Sep 21, 2024
@DaniPopes DaniPopes deleted the dani/evm-dyn-inspector-1 branch September 21, 2024 14:48
rplusq pushed a commit to rplusq/foundry that referenced this pull request Sep 25, 2024
ludamad added a commit to AztecProtocol/aztec-packages that referenced this pull request Feb 9, 2026
…port

Replace inline retry logic and direct forge calls with a standalone
TypeScript wrapper script (l1-contracts/scripts/forge_broadcast.ts).

Key behaviors:
- --batch-size 8 to prevent forge broadcast hangs
- External timeout (forge's --timeout is unreliable for broadcast hangs)
- On anvil: detect via web3_clientVersion, retry from scratch on failure
  (anvil's auto-miner race condition strands txs in mempool)
- On real chains: retry with --resume to pick up unmined transactions
- Buffers stdout per attempt, only emits the successful attempt's output
- Waits for stdout drain before exit to avoid pipe truncation

References:
- foundry-rs/foundry#6796 (batch size hang)
- foundry-rs/foundry#8919 (anvil auto-miner race)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants